src/main/java/com/example/solidconnection/mentor/domain/Mentor.java (1)

56-68: 생성자의 universityId 파라미터 타입 불일치에 주의하세요.

1. 생성자 파라미터는 Long universityId (박싱 타입, line 60)인데, 필드는 long universityId (원시 타입, line 46)입니다.
2. 만약 null이 전달되면 auto-unboxing 시 NullPointerException이 발생합니다.

학습된 비즈니스 규칙에 따르면 APPROVED 상태의 MentorApplication은 항상 non-null universityId를 가지므로 현재 흐름에서는 문제가 없지만, 방어적 코딩 관점에서 파라미터 타입을 long으로 통일하거나, null 체크를 추가하는 것을 고려해 주세요. Based on learnings, APPROVED 상태의 MentorApplication은 항상 non-null universityId를 가진다는 비즈니스 규칙이 있습니다.

💡 파라미터 타입 통일 제안

     public Mentor(
             String introduction,
             String passTip,
             long siteUserId,
-            Long universityId,
+            long universityId,
             long termId
     ) {

src/main/java/com/example/solidconnection/admin/service/AdminMentorApplicationService.java (1)

48-67: 멘토 생성 로직이 잘 구현되어 있습니다. 다만 검증 순서를 개선하면 더 안전합니다.

현재 흐름:

1. mentorApplication.approve() (상태 변경) → line 51
2. validateUserCanCreateMentor() (중복 검증) → line 55

트랜잭션 롤백으로 인해 실제 데이터 무결성 문제는 없지만, approve() 이전에 모든 사전 검증을 완료하는 것이 방어적 프로그래밍 관점에서 더 안전합니다. 만약 향후 approve() 내부에 이벤트 발행 등의 부수 효과가 추가된다면, 롤백만으로는 충분하지 않을 수 있습니다.

🔄 검증 순서 변경 제안

     `@Transactional`
     public void approveMentorApplication(Long mentorApplicationId) {
         MentorApplication mentorApplication = mentorApplicationRepository.findById(mentorApplicationId)
                 .orElseThrow(() -> new CustomException(MENTOR_APPLICATION_NOT_FOUND));
-        mentorApplication.approve();
 
         SiteUser siteUser = siteUserRepository.findById(mentorApplication.getSiteUserId())
                 .orElseThrow(() -> new CustomException(USER_NOT_FOUND));
         validateUserCanCreateMentor(siteUser.getId());
 
+        mentorApplication.approve();
         siteUser.becomeMentor();
         Mentor mentor = new Mentor(
                 null,
                 null,
                 siteUser.getId(),
                 mentorApplication.getUniversityId(),
                 mentorApplication.getTermId()
         );
 
         mentorRepository.save(mentor);
     }